Revert "file chooser: Allow activating without double-click"

This reverts commit fb0a13b7f070a14312dafa1e4df6ba03cf33be01.

It is getting in the way of gesture conversion, and didn't
really make anybody happy anyway.

app chooser: Use a gesture

Instead of connecting to ::button-press-event, use
a multipress gesture here.

Remove a leftover icon-size property

This made the 'style classes' demo crash.

Drop ::focus-in/out-event

These signals are no longer used in GTK+.
If you need to catch focus events, use the ::event signal.

treeview: Stop using ::focus-out-event

We can just use the generic ::event signal.

file chooser entry: Stop using ::focus-out-event

Just use the generic ::event signal.

spin button: Stop using ::focus-out-event

Just use the generic ::event signal.

calendar: Stop using ::focus-out-event

Use the generic ::event signal.

color editor: stop using ::focus-out-event

Again, the has-focus property is sufficient.

text cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

spin cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

combo cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

tests: Stop using ::focus-in/out-event

The has-focus property works just as well here.

popover: Stop using ::focus-in/out-event

We can use the is-active property of the window instead.

application: Stop using ::focus-in-event

We are really interested in the active window, so use
that property directly.

window: Stop using ::focus-in/out-event

We can just use the generic ::event.

text view: Stop using ::focus-in/out-event

We can just use the generic ::event here.

entry completion: Stop using ::focus-out-event

We can just use ::event.

entry: Stop using ::focus-in/out-event

We can just use the generic ::event signal.

a11y: Stop using ::focus-in/out-event

We can just use the generic ::event signal.

Drop gtk_widget_hide_on_delete

This function is misnamed - it is only ever relevant for
windows. And with the ::hide-on-delete property , it is
no longer necessary to use the signal for this simple case.

Drop ::event-after

This signal is not used in GTK+, and we are reducing the
amount of widget signals devoted to events in favor of
event controllers.

Drop ::grab-broken-event

This signal is no longer used in GTK+. Grab broken events
can be handled with the generic ::event signal.

button: Stop using ::grab-broken-event

The event-specific signals are going away. Just use ::event here.

menushell: Stop using ::grab-broken

The event-specific signals are going away. Just use ::event here.

notebook: some cleanup

notebook: Use a motion event controller

This lets us avoid the ::motion-notify-event handler.

textview: Use a motion event controller

This lets us replace the ::motion-notify-event handler.

Drop ::enter/leave-notify-event

These signals are no longer used in GTK+ and have a (not quite
perfect yet) replacement with GtkEventControllerMotion.

If you need to catch the raw events, the generic ::event
signal still works.

menu: Use a motion event controller

This lets us replace the ::enter/leave-notify-event handlers.
Not that there is a FIXME here - we currently can't get the
crossing mode from the event controller, so we have to fall
back to gtk_get_current_event().

menuitem: Use a motion event controller

This lets us replace the ::enter/leave-notify-event handlers.
Not that there is a FIXME here - we currently can't get the
crossing mode from the event controller, so we have to fall
back to gtk_get_current_event().

iconview: Use a motion event controller

This can replace ::motion-notify-event and ::leave-notify-event.

scrolled window: Use a motion event controller

This lets use replace one of the last uses of ::leave-notify-event.

entry completion: Drop initial selection avoidance

This is a minor thing, and it is hard to recreate
with event controllers, so drop it for now.

treeview: Use a motion event controller

This replaces handlers for ::enter/leave/motion-notify-event.

Add x/y to GtkEventControllerMotion::enter

We have this information available in enter events,
and having it in the controller signal as well makes
porting easier.

Update existing users.

Drop ::configure-event

This signal is unused in GTK+ and configure events are handled
internally by GtkWindow.

If you need to catch configure events, the generic ::event
signal still works.

window: Stop using ::configure-event

We already have a generic ::event handler that we
can just press into service for this.

testgtk: Stop using ::configure-event

xim: Stop using ::configure-event

The generic ::event will do.

tests: Stop using ::configure-event

The generic ::event will do.

Drop ::touch-event

This signal is not used in GTK+, and has a suitable
replacement with GtkGesture and its subclasses.

If you need to catch the raw touch events, the generic ::event
signal still works, too.

Drop ::scroll-event

This signal is not used in GTK+, and has a suitable
replacement with GtkEventControllerScroll.

If you need to catch these events, the generic ::event
signal still works, too.

Drop ::proximity-in-event and ::proximity-out-event

These are unused in GTK+, and are not very common.
If you need to catch these events, the generic ::event
signal still works.

Drop ::map-event and ::unmap-event

These are unused in GTK+, and are not very common.
If you need to catch these events, the generic ::event
signal still works.

window: Stop using ::map-event

Use ::event here, since we want to drop the event-specific
signals, and we already have a handler for the generic signal.

widget: Stop using ::map-event

Use ::event here, since we want to drop the event-specific
signals.

Drop GtkWidget::destroy-event

This signal is unused in GTK+, and is documented as
'hard to ever get', so there's no good reason for
keeping it.

Drop GtkWidget::delete-event

Nobody is using this signal anymore, and ::delete is there
for the few places where it is needed.

Replace all remaining uses of delete-event

Replace GtkWidget::delete-event by GtkWindow::close-request

The event is not useful at all, so we are better off
with a signal that doesn't have it, and it is only
relevant on toplevel windows, so we don't need it on
GtkWidget.

With this commit, delete events no longer go through the
::event, ::delete-event, ::event-after widget signals,
but just cause the ::close-request signal on GtkWindow to be
emitted.

testoverlay: Use an event controller

We can stop using ::enter-notify-event here, and use
our new motion event controller instead.

testgtk: Redo the event watcher case

Make this watch for emissions of ::event, since we
want to get rid of individual signals first.

testgtk: Use an event controller for 'testing scrolling'

The current code was not working with smooth scroll events,
for starters.

testinput: Drop some no-op signal handlers

These were not doing anything.

treeview: Don't handle ::delete-event

I don't think there is a way to get a delete event
on this popup - there's no window decorations, no close
button, etc. So no need to handle ::delete-event.

font chooser: Stop using ::scroll-event

We have an event controller for this purpose now.

treeview: Use a scroll event controller

This is the recommended way to deal with scroll events
now.

tests: Avoid a use of ::map-event

We can use the ::map signal here instead.

gtk-demo: Drop a use of ::event-after

It works just fine with ::event as well.

Drop gtk_true and gtk_false

These functions are entirely trivial, their documentation
is much longer than their implementation, and it contains
an example that is annotated as "don't do this"...

inspector: Stop using ::delete-event signal

We can just use GtkWindow::hide-on-close instead.

icon-browser: Stop connecting to ::delete-event

We can just use GtkWindow::hide-on-delete

widget-factory: Stop connecting to ::delete-event

We can use GtkWindow::hide-on-close instead

gtk-demo: Stop using ::delete-event

tests: Stop using the delete-event signal

We can use ::destroy in most cases.

treeview: Stop using gtk_true

doc tools: Stop using gtk_true

dialog: Stop using the ::delete-event signal

We can achieve the desired result with a class handler
and the ::hide-on-close property.

file chooser native: Stop connecting to ::delete-event

We can just use GtkWindow::hide-on-close.

file chooser button: Stop connecting to ::delete-event

We can just use GtkWindow::hide-on-close.

applicationwindow: Stop connecting to ::delete-event

We can just use GtkWindow::hide-on-close.

about dialog: Stop connecting to ::delete-event

We can just use GtkWindow::hide-on-close.

color button: Stop connecting to ::delete-event

We can just use GtkWindow::hide-on-close.

font button: Use GtkWindow::hide-on-close

Instead of the ::delete-event signal.

window: Add a hide-on-close property

This lets us avoid ::delete-event signal handlers for just
this purpose.

Update Polish translation

inspector: Avoid a critical

I forgot to initialized the text field when I added
it back.

vulkan: use GDK_DISPLAY_NOTE

Where we have a display, we should use the per-display logging.

vulkan: Fix build

This was changed in e151058dffbfd3b43798c2e6813eae9b952ba2d1 but
GDK_NOTE only takes 2 arguments.

tests: Update gl outset shadow tests

The results are slightly different for these now.

gl renderer: Render unblurred outset shadows differently

We don't need to draw anything to a texure for those.

gskcairoblur: Don't apply y_scale twice

A version of 29f36fed085d51bad16ec0e3f7ba5085bf9f82e8 but for the blur
we use in gsk.

Update docs for debug cleanup

Drop mention of GDK_GL, GDK_VULKAN and GDK_RENDERING_MODE,
add docs for GSK_DEBUG and new debug keys that were added.

inspector: Some logging support

Add a way to toggle debug output on and off
from the inspector. For now, we don't add a
log viewer here, since that has the risk of
deadlock until we've the logging completely
separated by display, and also requires us
to install a log writer function, which
libraries are not supposed to do.

inspector: Turn off gsk debugging

We don't want debug spew from the renderer that
is used for the inspector window, so turn it off.

gsk: Add a setter for debug flags

We need to set the global flags, since these are picked
up initially by new renderers.

gsk: Reorganize env vars

Get rid of GSK_RENDERING_MODE and add the flags to GSK_DEBUG,
following the same pattern we use in gdk now.

gtk Use per-renderer flags for node names

We are moving to per-renderer flags, so lets check them here.

gsk: make logging per-renderer

Add a setter for per-renderer debug flags, and use
them where possible. Some places don't have easy access
to a renderer, so this is not complete.
Also, use g_message instead of g_print throughout.

gsk: Drop unused debug flags

We are not logging anything for transforms or rendernodes atm.

gsk: Clarify a debug message

We are printing a window type, not a display.

wayland: Use g_message for logging

g_printerr is not the best for this.

Drop the cairo-recording debug flag

It is not very useful. The cairo-image option on the
other hand is has been useful in tracking down problems
in the past, so we'll keep it.

Drop the gl-always debug option

This doesn't seem very useful.

Make gdk logging per-display

As far as possible, use per-display debug flags.

This will minimize the debug spew that we get from
the inspector if it is running on a separate display.

gdk: Reorganize env vars

Drop GDK_GL, GDK_VULKAN and GDK_RENDERING_MODE.
Merge the useful bits into GDK_DEBUG.
Drop unused debug flags (CURSOR).

Make geometry logging per-display

Use the new macro to do this.

icon theme: Make logging per-display

As far as possible, for now. This needs some more
work to cover all locations.

gtk: add a per-display log macro

This makes the conversion easier.